feat(runtime-host): discover coordination relays automatically - #4126
Conversation
55aaf3e to
5d170ee
Compare
Use a bounded IPFS Amino discovery client to supply public Circuit Relay v2 candidates while the main peer endpoint retains reservation authority. Preserve manual relay precedence and publish only accepted reservations. Generated-by: OpenAI Codex
5d170ee to
61c56bd
Compare
jackwener
left a comment
There was a problem hiding this comment.
Approved at exact head 61c56bd4a52b2c29c74e0347d3bfd368b99c2a86; I found no P0–P2 issues.
I traced the discovery path from the bounded, client-only DHT swarm through relay probing, reservation acceptance, and signed Peer Mesh route publication. The main peer endpoint remains the reservation authority, manually configured relays count first toward the target, and only validated addresses from accepted reservations enter the published route set. The implementation also provides Desktop and CLI opt-out controls while keeping the legacy peer-management frame compatible through an optional field and capability-gated status request.
Verification passed on the exact head: the root build and 60 focused Runtime Host, CLI, and Desktop tests completed successfully, and Rust formatting is clean. Local Rust tests could not run because this machine has Rust 1.95 while the crate requires 1.98; the exact-head hosted native-addon builds, test, quality, package, windows_recovery, and installed-CLI validations all completed successfully (the Eval validation job was skipped by its workflow). A synthetic merge with current main at c314e9eb0b96e10abfa35f10965e8cc9d2fc7b9e is clean and its full root build passes.
Posted by an automated review agent operated by @WAWQAQ. This is not an
independent human review and does not satisfy the committer review required by
CONTRIBUTING.md. A human is accountable for this comment — please push back if
anything here is wrong.
简体中文
本条评论由 @WAWQAQ 运行的自动化审查程序发出。它不构成 CONTRIBUTING.md
所要求的独立人类审查,也不能替代人类审查。有人类对本条评论负责,如有错误请直接指出。
English
Summary
Refs #3842
Verification
UI
AI use
Select exactly one:
Tool(s) and scope: OpenAI Codex implemented and verified the change under M4n5ter's direction and review
Checklist
Does this PR entail a change in behavior?
简体中文
摘要
关联 #3842
验证
界面
AI 使用
OpenAI Codex 在 M4n5ter 的指导和审核下实现并验证了本次变更
检查清单
本 PR 是否改变行为?